Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release] logzio APM Collector 1.0.0 #561

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

8naama
Copy link
Contributor

@8naama 8naama commented Nov 19, 2024

Description

As part of the Unified helm chart, this is the new chart for APM

  • Changes from logzio-telemetry
    • drop support for daemonset mode. In logzio-telemetry it's required for metrics scraping per node which is not needed for traces
    • support enabling autoscaling for SPM via VPA
    • Secret values are read from .Values.global.<secretParam> section. This change provides two key benefits:
      • Enables the definition of tokens in the parent chart without duplication
      • Preserves the ability of each sub chart to control the values of its secrets
    • OTEL configuration changes:
      • Merge the config of latency and calls which renames span.name >> operation
      • Add _seconds prefix to Service Graph metrics (which was dropped in v109) following product request to keep the naming of metrics as it was
      • Add connector.spanmetrics.legacyMetricNames feature gate to keep old names of SPM metrics (which were changed to match the Service Graph metrics names in v109) following product request
    • Fine tune ClusterRole permissions and drop cluster-admin role permissions
    • Add support for easily changing the OTEL Collector logging level via a flag otelLogLevel
    • Listener address for SPM is automatically generated based on the region, for better user experience
    • Move all custom configurations to be generated in .tpl files (_helpers and _config)
    • Add resource limitations to the pods
  • Aligned relevant tests
  • Added readme
    • add custom tail sampling explanation section
    • moved changelog to it's own file CHANGELOG.md to keep the readme more clean

What type of PR is this?

(check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build / CI
  • ⏩ Revert

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help from somebody

@8naama 8naama mentioned this pull request Nov 19, 2024
10 tasks
@8naama 8naama force-pushed the release/logzio-apm-collector-1.0.0 branch from 35f9f7e to 6796009 Compare December 10, 2024 11:56
@8naama 8naama marked this pull request as ready for review December 12, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant